You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fixes remote-to-local fallback so workspace show homeboy and workspace worktree add homeboy ... can reuse an existing local primary when remote workspace state is present but missing that repo.
Normalizes workspace worktree add URL/path repo arguments to a matching local primary, and rejects unsupported URL/path arguments instead of creating malformed github://https://... or github:///Users/... handles.
Bounds cleanup git worktree remove calls and removes the unbounded rm -rf fallback so cleanup apply rows fail clearly instead of wedging on a single worktree.
Adds focused smokes for listed-primary fallback, remote URL/path rejection, local URL/path normalization, and cleanup removal guardrails.
php tests/smoke-worktree-cleanup.php(fails: existing duplicate skip-row count drift in dirty/artifact/external summary assertions; removal execution and primary/outside safety assertions pass)
AI assistance
AI assistance: Yes
Tool(s): OpenCode (openai/gpt-5.5)
Used for: Drafted the workspace resolution fix, cleanup removal guard, regression smokes, and local verification commands; Chris remains responsible for review and merge.
ℹ️ Auto-fix lint issues: homeboy refactor data-machine-code --from lint --write
ℹ️ Collect coverage: homeboy test data-machine-code --coverage
ℹ️ Save test baseline: homeboy test data-machine-code --baseline
ℹ️ Pass args to test runner: homeboy test -- [args]
ℹ️ Full options: homeboy docs commands/test
Deep dive: homeboy test data-machine-code --changed-since 4057301
Artifacts and drill-down
CI results artifact: homeboy-ci-results-data-machine-code-test-quality-Linux-node24 contains immediate command JSON for this action invocation.
Observation artifact: homeboy-observations-data-machine-code-test-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
workspace show homeboyandworkspace worktree add homeboy ...can reuse an existing local primary when remote workspace state is present but missing that repo.workspace worktree addURL/path repo arguments to a matching local primary, and rejects unsupported URL/path arguments instead of creating malformedgithub://https://...orgithub:///Users/...handles.git worktree removecalls and removes the unboundedrm -rffallback so cleanup apply rows fail clearly instead of wedging on a single worktree.Fixes #637.
Fixes #638.
Tests
php -l inc/Workspace/WorkspaceWorktreeCleanupEngine.phpphp -l tests/smoke-worktree-cleanup-remove-guard.phpphp tests/smoke-workspace-local-fallback.phpphp tests/smoke-worktree-cleanup-remove-guard.phpphp tests/smoke-worktree-cleanup.php(fails: existing duplicate skip-row count drift in dirty/artifact/external summary assertions; removal execution and primary/outside safety assertions pass)AI assistance